Skip to content

Conversation

@josephfusco
Copy link
Member

This pull request updates the workflow for posting plugin artifact links on pull requests. The main change ensures that the bot checks for an existing comment and updates it instead of creating a new one each time, improving comment management and reducing duplication.

Enhancements to comment management:

  • .github/workflows/plugin-artifact-for-pr.yml: Added logic to find existing comments from the bot and update them if they already mention the plugin artifact, instead of creating new comments. This prevents duplicate comments and keeps the pull request cleaner.

download

@josephfusco josephfusco changed the title Update plugin-artifact-for-pr.yml ci: Improve plugin artifact bot to reuse comments in PRs Jun 12, 2025
@josephfusco josephfusco requested a review from Copilot June 12, 2025 16:38
@colinmurphy
Copy link
Member

Thanks @josephfusco

That's a really good idea :) I had noticed there was too many comments so thanks for this :)

@wpengine wpengine deleted a comment from github-actions bot Jun 12, 2025

This comment was marked as outdated.

@github-project-automation github-project-automation bot moved this to 🆕 Backlog in Headless OSS Jun 12, 2025
@josephfusco josephfusco moved this from 🆕 Backlog to 👀 In review in Headless OSS Jun 12, 2025
@josephfusco josephfusco marked this pull request as ready for review June 12, 2025 17:11
@josephfusco josephfusco requested a review from a team as a code owner June 12, 2025 17:11
@josephfusco josephfusco enabled auto-merge June 12, 2025 17:12
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR improves the plugin artifact bot, ensuring that it reuses existing comments instead of creating duplicate ones when posting plugin artifact links on pull requests.

  • Updated the workflow in .github/workflows/plugin-artifact-for-pr.yml to search for and update an existing bot comment.
  • Added a testing comment in README.md to help trigger the workflow during testing.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
plugins/wp-graphql-headless-webhooks/README.md Added a testing comment for triggering the workflow (temporary).
.github/workflows/plugin-artifact-for-pr.yml Updated comment posting logic to reuse existing bot comments.
Comments suppressed due to low confidence (1)

plugins/wp-graphql-headless-webhooks/README.md:2

  • [nitpick] Consider removing or updating the testing comment once workflow testing is complete to avoid leaving temporary code in production documentation.
<!-- Test to trigger workflow - can be removed after testing -->

@wpengine wpengine deleted a comment from github-actions bot Jun 12, 2025
@github-actions
Copy link

github-actions bot commented Jun 12, 2025

ℹ️ Download the latest wp-graphql-headless-webhooks plugin zip from this PR
(See the 'Artifacts' section at the bottom)

@wpengine wpengine deleted a comment from github-actions bot Jun 12, 2025
@wpengine wpengine deleted a comment from github-actions bot Jun 12, 2025
@josephfusco
Copy link
Member Author

@colinmurphy this can be seen working here #252 (comment)

🙌🏼

colinmurphy
colinmurphy previously approved these changes Jun 12, 2025
Copy link
Member

@colinmurphy colinmurphy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@josephfusco LGTM 🚀 🚀 🚀

@josephfusco josephfusco disabled auto-merge June 12, 2025 17:27
@josephfusco josephfusco added this pull request to the merge queue Jun 12, 2025
Merged via the queue into main with commit f6bbf64 Jun 12, 2025
1 check passed
@josephfusco josephfusco deleted the consolidate-pr-workflow-comments branch June 12, 2025 17:42
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Closed in Headless OSS Jun 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Closed

Development

Successfully merging this pull request may close these issues.

3 participants